Skip to content

feat(CDPSDK-2436): add CreateSubscription for Webhook Events#634

Open
milan-cb wants to merge 5 commits intomainfrom
milan/CDPSDK-2436
Open

feat(CDPSDK-2436): add CreateSubscription for Webhook Events#634
milan-cb wants to merge 5 commits intomainfrom
milan/CDPSDK-2436

Conversation

@milan-cb
Copy link
Copy Markdown
Contributor

@milan-cb milan-cb commented Apr 2, 2026

Description

This PR adds the CreateSubscription method to the Webhooks client.

Tests

Unit tests have been added for this new method. In addition, this has been tested in dev.

Checklist

A couple of things to include in your PR for completeness:

  • Updated the typescript README if relevant
  • Updated the python README if relevant
  • Added a changelog entry
  • Added e2e tests if introducing new functionality

@linear
Copy link
Copy Markdown

linear bot commented Apr 2, 2026

@cb-heimdall
Copy link
Copy Markdown

cb-heimdall commented Apr 2, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@milan-cb milan-cb requested a review from sddioulde April 2, 2026 20:51
Copy link
Copy Markdown
Contributor

@sddioulde sddioulde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. i think we should add a test from the webhooks client perspective so that we are sure it is correctly wired to the action

@milan-cb milan-cb marked this pull request as ready for review April 7, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants